fix c++11 range-loop might detach Qt container
authortsteven4 <13596209+tsteven4@users.noreply.github.com>
Wed, 20 Nov 2019 18:21:05 +0000 (11:21 -0700)
committertsteven4 <13596209+tsteven4@users.noreply.github.com>
Wed, 20 Nov 2019 18:21:05 +0000 (11:21 -0700)
found with clazy 1.6

util.cc
xcsv.cc

diff --git a/util.cc b/util.cc
index f080a353f008b1a863b3fa0a14a4ba09d4368c8c..41db0a478faaeb66a1959b9a846860dfc8637876 100644 (file)
--- a/util.cc
+++ b/util.cc
@@ -1734,7 +1734,7 @@ list_codecs()
 {
   QTextStream info(stderr);
   info.setFieldAlignment(QTextStream::AlignLeft);
-  auto mibs = QTextCodec::availableMibs();
+  const auto mibs = QTextCodec::availableMibs();
   int maxlen = 0;
   for (auto mib : mibs) {
     auto codec = QTextCodec::codecForMib(mib);
diff --git a/xcsv.cc b/xcsv.cc
index e6cd5bee326b78c90d1223d6bc6b7521d58750a3..20ee0d5e5fb1e42aca63dd52788440aca536b7fa 100644 (file)
--- a/xcsv.cc
+++ b/xcsv.cc
@@ -2014,7 +2014,7 @@ xcsv_parse_style_line(QString line)
 static void
 xcsv_parse_style_buff(const char* sbuff)
 {
-  QStringList lines = QString(sbuff).split('\n');
+  const QStringList lines = QString(sbuff).split('\n');
   for (const auto& line : lines) {
     xcsv_parse_style_line(line);
   }